Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

remote usage: POC!! #86

Closed
wants to merge 43 commits into from
Closed

remote usage: POC!! #86

wants to merge 43 commits into from

Conversation

bounceme
Copy link
Contributor

@bounceme bounceme commented Jul 12, 2017

@bounceme
Copy link
Contributor Author

bounceme commented Jul 13, 2017

actually can just use cURL for this! will update later

@bounceme bounceme changed the title remote lynx usage: POC experimental!! remote usage: POC experimental!! Jul 13, 2017
@bounceme
Copy link
Contributor Author

basically just curl [url] -X NLST or LIST

@hauleth
Copy link

hauleth commented Jul 13, 2017

Personally I would like to see it as an additional plugin that would extend dirvish instead of adding such functionality directly.

@bounceme
Copy link
Contributor Author

mostly just opened a pr for advice/feedback. don't really think this will ever mature at all

@bounceme
Copy link
Contributor Author

bounceme commented Jul 13, 2017

@hauleth
Copy link

hauleth commented Jul 14, 2017

Personally I still suggest anyone who need such use case to use FUSE:

Alternatively you can use any file watcher (like watchman) and rsync/scp files to the remote.

Both solutions should be easier more error prone than extending dirvish.

@bounceme bounceme closed this Jul 14, 2017
@justinmk
Copy link
Owner

@bounceme Thanks for the ideas here. If you have any further insights do share them. As long as #62 is open it's worth collecting ideas. Don't know what will come of it, but if there's a simple way to reach an 80% solution it might be worth it.

@bounceme bounceme reopened this Jul 15, 2017
@bounceme bounceme changed the title remote usage: POC experimental!! remote usage: POC dont merge!! Jul 15, 2017
@bounceme bounceme changed the title remote usage: POC dont merge!! remote usage: POC!! Jul 15, 2017
@bounceme
Copy link
Contributor Author

bounceme commented Jul 16, 2017

@justinmk need some help with this: 96a17f6#diff-9bbb7c6a1278ae4fda0476d24641162aR291 . without the change here i was experiencing complete vim hangup from sourcing many irrelevant filetypes syntax files. maybe it has to do with :h shada-% which i use: set viminfo^=%25. the problem happens when leaving a ftp:// directory, though maybe others are affected too (will test)

@bounceme
Copy link
Contributor Author

bounceme commented Jul 16, 2017

Seems to be related to not having a altbufname, e.g. 1 %a [No Name]

@bounceme
Copy link
Contributor Author

bounceme commented Jul 16, 2017

instead of a hang, this error appears, if you have completely disabled netrw with g:loaded_netrw

edit: and without my small change

Error detected while processing function <SNR>64_on_bufenter:
line    2:
E121: Undefined variable: b:dirvish
E116: Invalid arguments for function extend(get(w:, 'dirvish', {}), b:dirvish, 'keep')
E15: Invalid expression: extend(get(w:, 'dirvish', {}), b:dirvish, 'keep')

@bounceme
Copy link
Contributor Author

@justinmk https://github.com/justinmk/vim-dirvish/compare/master...bounceme:patch-1?w=1#diff-9bbb7c6a1278ae4fda0476d24641162aR405 isn't relevant to this pr, but allows an nameless buffer to be preserved, since edit will take it over instead of making another buffer. should that be a different p.r.?

@justinmk
Copy link
Owner

sure...

@bounceme
Copy link
Contributor Author

probably just going to forego this patch, repackaging it as a external collection of viml hacks, atleast so i can disable netrw.

@bounceme
Copy link
Contributor Author

scratch that last comment. the ugliness required to make something work, with all the altbuf, prevbuf, isdirectory/isreadable, code, needs a rewrite to be honest

@justinmk
Copy link
Owner

a rewrite ... ?

@bounceme
Copy link
Contributor Author

most functions currently have those things. so its not practical really, either to try and work around or make an api

@bounceme
Copy link
Contributor Author

don't worry, im not going to be starting a rewrite pr

@justinmk
Copy link
Owner

That's where the strategies approach would come in.

@bounceme
Copy link
Contributor Author

bounceme commented Oct 4, 2017

@bounceme bounceme closed this Oct 4, 2017
@justinmk
Copy link
Owner

justinmk commented Oct 4, 2017

@bounceme cool, I'll try it out.

@justinmk
Copy link
Owner

justinmk commented Oct 7, 2017

@bounceme that wouldn't work with ssh would it?

@bounceme
Copy link
Contributor Author

bounceme commented Oct 7, 2017

no, im working on it though haha. just need to comprehend dotglob, *, ls to make a parseable directory listing first

@bounceme
Copy link
Contributor Author

bounceme commented Oct 7, 2017

if curious, i think im going with { find . -type d -maxdepth 1 | sed -e 's/$/\//' && find . \! \( -type d \) -maxdepth 1 ; } | sed -e 's/^\.\///'

@bounceme
Copy link
Contributor Author

got it working with ssh! Only working with the netrw-ssh-hack method though, so there is no password input done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants